cell area: Use standard padding

aspect frame: Make final

assistant: Drop the priv pointer

assistant: Make final

accel label: Make final

about dialog: Make final

cell renderer pixbuf: Drop the priv pointer

cell renderer pixbuf: Make final

cell renderer progress: Drop the priv pointer

cell renderer progress: Make final

cell renderer spin: Drop the priv pointer

cell renderer spin: Make final

cell renderer spinner: Drop priv pointer

cell renderer spinner: Make final

cell renderer text: Drop the priv pointer

cell renderer text: Use standard padding

cell renderer toggle: Drop the priv pointer

cell renderer toggle: Make final

cell renderer combo: Drop the priv pointer

cell renderer combo: Make final

cell renderer accel: Drop the priv pointer

cell renderer accel: Make final

cell renderer: Use standard padding

calendar: Drop the priv pointer

calendar: Make final

spinner: Make final

progress bar: Make final

print operation: Drop the priv pointer

print operation: Use standard class padding

Rename gtkprintbackend.h to gtkprintbackendprivate.h

Follow the naming convention for private headers.

print job: Make final

No need to subclass this.

printer: Drop the priv pointer

printer: Make private

We want to subclass this in print backends,
but we don't want to allow out-of-tree subclassing.

Rename gtkprinter-private.h

Name this header gtkprinterprivate.h, following
our convention for private headers.

stack switcher: Make final

This should not be subclassed.

stack sidebar: Make final

This should not be subclassed.

print dialog: Drop the priv pointer

print dialog: Make final

Nobody should subclass this.

page setup dialog: Drop the priv pointer

page setup dialog: Make final

No subclassing needed.

switch: Make final

This should not be subclassed.

icon theme: Make final

Nobody should subclass GtkIconTheme or GtkIconInfo.

textview: Use standard padding

Given that GtkSourceView exists, we will have
to keep GtkTextView subclassable until a replacement
appears.

viewport: Cleanup

viewport: Make final

No need to derive from it.

Trivial

surface: fix up docs

The docs for gdk_surface_invalidate_region were
talking about a no-longer-existing argument.

color plane: Don't allocate 0-size textures

This triggers critials, and does us no good.

Cosmetics

rendernodeparser: Parse Cairo nodes

We also print the script, but we don't parse it yet.

rendernode: Remove unused function

inspector: Make updates overlay not crash

... when a window gets hidden and later reshown.

The code now properly cleans up a window when it gets unmapped instead
of trying to retain previous updates information.

Revert "wayland: Keep some resources until destroy"

This reverts commit f2d598b9a19cd7151c592e9c74d5b0d0ba3c9e48.

widget-factory: Add some animation in a popover

broadway: add frame times to the profiler

fishbowl: Add a menubutton demo

This reveals some issues with the position tracking.

menu button: Add popup/popdown functions

These are needed, after all.

fishbowl: Fix the lock button

gl renderer: Make creating render targets easier

gl renderer: Remove some unused api form the GLDriver

fishbowl: Don't show ridiculous precision

Two decimals are more than enough for the frame rate.

Merge branch 'issue1901-gtk4' into 'master'

wayland: Use scale from core wl_output protocol

Closes #1901

See merge request GNOME/gtk!870

wayland: Use scale from core wl_output protocol

The “xdg-output” protocol provides clients with the outputs size and
position in compositor coordinates, and does not provide the output
scale which is already provided by the core “wl_output” protocol.

So when receiving the wl_output scale event, we should update the scale
regardless of “xdg-output” support, otherwise the scale will remain to
its default value of 1 and the surface will be scaled up by the
compositor to match the actual output scale, which causes blurry fonts
and widgets.

Fixes: https://gitlab.gnome.org/GNOME/gtk/issues/1901
Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

gtk-demo: Reinstate flat menu buttons

widget-factory: Reinstate flat menu buttons

menu button: Add a relief property

This is used in various places.

testscrolledwindow: GtkMenuButton != GtkContainer

It's not anymore, so don't cast it to one... Use the new :label instead.

testsuite: Fix leak

Merge branch 'mkenums-once' into 'master'

Use g_once for all the mkenums template files

Closes #1617

See merge request GNOME/gtk!865

Use g_once for all the mkenums template files

Without this, initialization of enum/flags values from multithreaded
programs may not work correctly.

Fixes #1617.

Merge branch 'win32-enable-werror' into 'master'

Win32 enable werror

See merge request GNOME/gtk!861

Merge branch 'issue-302-master' into 'master'

docs: Fix the override for GdkAtom

Closes #302

See merge request GNOME/gtk!864

docs: Fix the override for GdkAtom

GdkAtom is a typedef to a pointer to an opaque structure. We need to
tell GTK-Doc how to override it, so that the documentation is accurate.

Fixes: #302

gl renderer: Render recording surface untransformed

Cairo can't handle the y flip apparently.

Fixes #1906

gl glyphcache: Don't choke on huge glyphs

We blindly assume everywhere that a single glyph will definitely fit on
one atlas, but that's not always the case.
For now, don't crash or produce GL errors.

widget: Don't always call get_display() when allocating

The GTK_DISPLAY_DEBUG_CHECK macro will cleverly only call the function
if any of the display debug flags are set, so in the common case it
won't even be executed.

win32: Fix meson names for defines

Those have been wrong forever, but nobody cares because they get to the
values we want anyway.

win32: Enable --werror in CI

win32: Remove outdated workaround

The function that was missing in 2016 does exist now.

Fixes gcc complaining about a wrong declaration.

tests: Use g_free(), not free()

Windows doesn't like that.

imcontextime: Add missing semicolon

I wonder how thoroughly this was tested... ;)

win32: Add missing enum value to switch statement

imcontextime: Remove unused variables

win32: Rename a shadowing variable

window: Remove unneeded variable

The variable would show up as a gcc warning in builds without X11
support.

win32: Actually pass y variable for y parameter

win32: Remove unused variables

Merge branch 'wip/otte/for-master' into 'master'

Wip/otte/for master

See merge request GNOME/gtk!860

testsuite: Remove serialize-deserialize test

The serialize-deserialize test is completely handled via the new
nodeparser tests.

So move those tests over and delete the old test binary.

testsuite: Add render tests for all the empty nodes

Test that rendering empty nodes succees. For a lot of nodes the
resulting rendering isn't clearly defined, in those cases we overdraw
those regions (sometimes the whole image) with black.

testuite: Improve compare-render binary

- Remove remains of g_test_*() functions
  We're not a glib test, we're a simple binary.
- Handle nonexistence of reference image properly
  Don't assert, but create the output image and the error out.

rendernode: Blur alpha channel for cairo blur node

The blur node still doesn't properly blur with a transform, but at least
it now doesn't turn semi-transparent stuff black.

testsuite: Add empty node tests

For every node type, add a test that creates it empty and ensure that
it prints the reference output.

rendernodeparser: Allow parsing strings

Instead of only allowing for glyph indexes, allow ASCII characters as
replacements. So this glyph sequence
  glyphs: 65 8, 66 8, 67 8
Can be replaced by
  glyphs: "ABC"
provided that the glyph for "A", "B" and "C" are 65, 66 and 67
respectively and their advance is exactly 8.
x offset and y offset must always be 0 and every glyph must start a
cluster.

rendernodeparser: Update to new rules

Update to the docs outlined in #1887.

In particular, the changes do:

1. Require no property, have a working default for everything
2. Be clear about what gets printed and how.

Tests ahve been adapted to still pass.

rgba: Add GDK_RGBA() macro

So far it's private, but it's a pretty cute way to use hex colors, so we
might conside making it public.

rendernode: Take a graphene_point_t for the offset

... instead of 2 floats.

testsuite: Redo node-parser

Base the rewrite on testsuite/css/parser/test-css-parser - we now
require the node file to match a reference node and track the errors it
triggers.
We also no longer use gtester.

rendernodeparser: Skip root node when it's a container

When printing, behave the same way as when parsing:
Magically skip a container node if there is one - just like the
parser magically creates a container node to hold all the nodes
it parses.

rendernodeparser: Allow single values instead of 4

This allows writing:
  colors: red;
instead of
  colors: red red red red;
to draw a red border.